Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Black Lives Matter banner #3089

Closed
wants to merge 1 commit into from
Closed

Conversation

Daniel15
Copy link
Member

@Daniel15 Daniel15 commented Jun 2, 2020

Similar to the Docusaurus and React sites.

image

React's version, for reference:

image

It seems harder to show it above the menubar with Docusaurus, so I placed it underneath instead. This is consistent with what the Docusaurus site itself is doing.

Also see:
reactjs/react.dev#3004
facebook/docusaurus#2862

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josephsavona has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@josephsavona
Copy link
Contributor

josephsavona commented Jun 5, 2020

Thank you for sending this PR. We're in the process of importing and landing this change, so it should (barring hiccups) roll out to the website this evening.

We're actively discussing how our team can go beyond simply expressing support and take action to support progress on racial justice. In the meantime, we're supportive of adding this banner to help raise awareness of this issue.

@facebook-github-bot
Copy link
Contributor

@josephsavona merged this pull request in 3d3aa59.

@Daniel15
Copy link
Member Author

Daniel15 commented Jun 6, 2020

@josephsavona It seems like the styling isn't working correctly on the live site :/ https://relay.dev/

Never mind, it was an old cached version. Cleared my cache and it's OK now.

@Daniel15 Daniel15 deleted the blm branch December 16, 2022 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants